-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ADD] SFU server #1
Conversation
271ee6d
to
08f63d2
Compare
08f63d2
to
209af28
Compare
The SFU is a server used to relay streaming between the participants of a call to provide better scalability and performance than the current peer-to-peer connections. [README](https://github.com/odoo/sfu/tree/master-discuss-rtc-server2-tso/) Old PR: odoo/enterprise#45861 Related to: odoo/odoo#132153 task-2765922
209af28
to
0c61202
Compare
"node": true, | ||
"browser": false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isn't node or browser differently true or false (or sometimes both at the same time) depending on the file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the overrides below are setting the env with the right values for the few files where it is not (just) node
|
||
- GET `/v1/stats`: returns the server statistics in JSON: | ||
```json | ||
[ // one array entry per channel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment is invalid json, suggestion to use a specific key "_comment" or comment it separately
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix for both README comments in #2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we reached the point where we should just merge and fix if necessary. Keeping the full diff pending hoping to reach perfection becomes counter-productive now.
The SFU is a server used to relay streaming between the participants of a call to provide better scalability and performance than the current peer-to-peer connections.
README
Old PR: https://github.com/odoo/enterprise/pull/45861
Related to: odoo/odoo#132153
task-2765922